Skip to content

Update main to latest version for Feb 5, 2025, training and workshop#12

Merged
jacob-a-brown merged 59 commits into
mainfrom
pre-production
Feb 4, 2025
Merged

Update main to latest version for Feb 5, 2025, training and workshop#12
jacob-a-brown merged 59 commits into
mainfrom
pre-production

Conversation

@jacob-a-brown

Copy link
Copy Markdown
Contributor

No description provided.

jacob-a-brown and others added 30 commits January 13, 2025 11:28
Some sources were listed under ST2 and CKAN. This change explicitly names
the individual sources. This will help with development because some
organizations plan to add data to CKAN (Santa Fe expressed interest). So
by explicitly naming the sources, we can better communicate to users
where the data is coming from.

Also, each source now has a link to where the data is hosted.
…comes from in config report

There are some sources that only report water levels or analyte
measurements. Prior to this update, for example, if the user requested
water level data, the sources that only reported analyte measurements
would say that data was being pulled from them. Now, the sources that
only report water level or analyte measurements are separated so that
the user can see where the data is coming from.
More sources will be added to CKAN, so it will no longer only be NM OSE Roswell
data. So, NM OSE Roswell should be excluded/included based on the source,
not that the data is hosted on CKAN.
Source names have been updated to indicate the organization and not
where the data is hosted (such as CKAN or ST2). State agencies have
also been prepended to OSE Roswell, ISC Seven Rivers, AMP, and DWB for
clarity.

Set source to False if the source is not in use. With this change, however,
the user can still set `--no-{source}` even if it doesn't report that
parameter. This will prevent unecessary crashes
This enables location data to be easily determined for all sites
Even if a site has only one measurement, its data will be in a
standalone csv to have exports be standardized and findable
Saving the code for now in case we want to add it back
The user must now specify --output summary, --output timeseried_unfied,
or --output timeseries_separated to explicitly request the output format
This change was done to have the output type and the functions that write
the outputs have corresponding names
catalogItems are no longer returned in the location response. They are,
however, still necessary to retrieve data and records. They are now
explicitly found by querying the associated catalogRecords endpoint
This is done to standardize the way queries are called. This will allow
for easier use of the command line interface and will make the code more
readable.
This allows the user to call `weave {parameter name}` for ease of use,
rather than `weave parameter {parameter name}`
Now weave is called with `die weave {parameter}` instead of `weave {parameter}`
so that `sources` and `wells` are die commands.
This is done so that the tables are the same for both analytes and
waterlevels. The information isn't changing, but the two table now
have the same column headers
jacob-a-brown and others added 29 commits January 28, 2025 16:29
This is to get rid of floating point errors and keep results
consistent with old data
This is done so that outputs don't override each other. All outputs are
now found in directories titled `output`. If the output directory already
exists, the output will be placed in a directory with the same name but
with a number appended to the end. For example, if the output directory
`output` already exists, the output will be placed in `output_1`. If
`output_1` already exists, the output will be placed in `output_2`, and so
on.

die.log is also placed in the output directory so that each tool call
is recorded in a single log file.
Before this only the DIE only accounted for results with "< MRL". If
"< MDL" was in the result it would crash
This will make it so when the user invokes `die sources` the name
will be printed instead of the object's memory location.
This commit changes the parameter options to lowercase for consistency
and to make the code easier to maintain since we won't have to worry
about case sensitivity. The inputs are still case insensitive
Waterlevels and analytes are now included in parameters and their
export tables are now the same
convert_units returns the converted results and a warning message, so
use the first element of the returned tuple
Updated string padding for long names | publish to pypi on merges to Main
@jacob-a-brown jacob-a-brown merged commit 95fe83a into main Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant